Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Add edge styles support for DOT lib #3348

Merged
merged 3 commits into from
Aug 18, 2017
Merged

Conversation

yasufum
Copy link
Contributor

@yasufum yasufum commented Aug 7, 2017

Update for TODO: support of solid/dotted/dashed edges (attr = 'style') in lib/network/dotparser.js

dotparser.js does not support edge styles and this feature is described as TODO

I have added edge styles (solid, dashed and dotted) and an example examples/network/data/dotLanguage/dotEdgeSytles.html

Changes


  • Update dotparser.js
    Add edge style definition in parseAttributeList().

  • Create dotEdgeSytles.html
    For testing edge styles. It is based on examples/network/data/dotLanguage/dotPlayground.html.

yasufum added 2 commits August 7, 2017 18:57
* Update for TODO: support of solid/dotted/dashed edges (attr = 'style')
* Edge styles are defined as var edgeStyles in parseAttributeList().
Add an editable example for playing DOT edge styles
@bradh bradh added the Network label Aug 7, 2017
@yotamberk yotamberk requested a review from wimrijnders August 8, 2017 18:35
Copy link
Contributor

@wimrijnders wimrijnders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work! I especially like the way you leveraged existing functionality and kept your changes small. And 👍 for example.

Just the following:

  • Typo in example name: Sytles -> Styles
  • Remove the line with the TODO you tackled (around line 902)

* Correct typo of filename from 'dotEdgeSytles.html' to
  'dotEdgeStyles.html'.
* Remove TODO description of the issue for edge style
@yasufum
Copy link
Contributor Author

yasufum commented Aug 17, 2017

Thank you for reviewing and comments! I fixed it and please check my update.

Copy link
Contributor

@wimrijnders wimrijnders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect. Thank you warmly for your contribution.

@yotamberk yotamberk merged commit e44b0b0 into almende:develop Aug 18, 2017
primozs pushed a commit to primozs/vis that referenced this pull request Jan 3, 2019
* Add support for edge style

* Update for TODO: support of solid/dotted/dashed edges (attr = 'style')
* Edge styles are defined as var edgeStyles in parseAttributeList().

* Add example for edge styles

Add an editable example for playing DOT edge styles

* Correct typo and remove TODO description

* Correct typo of filename from 'dotEdgeSytles.html' to
  'dotEdgeStyles.html'.
* Remove TODO description of the issue for edge style
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants